cfg.mk: Add a syntax check for a redundant : in glnx_throw

This is going to be an easy bug to make when porting.

Example spotted by jlebon in https://github.com/ostreedev/ostree/pull/752/files/435c2dc35c06218b8fe5b4ede0fe4f6cda535b45#diff-dbc2359685171239c4708f2a8f6aa921

Closes: #752
Approved by: jlebon

diff --git a/cfg.mk b/cfg.mk
index d6a5039d26d36921da238748d5a5a8ef63aa3c3f..b4ffaf5f55fdccb33433c5455af3db2a5027247a 100644 (file)
--- a/cfg.mk
+++ b/cfg.mk
@@ -23,6 +23,10 @@ local-checks-to-skip = \
     sc_prohibit_path_max_allocation \
     sc_trailing_blank \
 
+sc_glnx_errno_prefix_colon:
+       @prohibit='\<glnx_throw_errno_prefix *\(.*: ",' halt="don't add trailing : for glnx_throw_errno_prefix" \
+         $(_sc_search_regexp)
+
 #SHELL=bash -x
 show-vc-list-except:
        @$(VC_LIST_EXCEPT)